Good one, thanks for finding this bug!

Maybe I'm asking for too much but could you point to the code in Puppet that generates the SSL certs, to make sure they always are created in lowercase?

FQDNs are case insensitive from my 5 min investigation and fiddling with /etc/hosts, so it's not an issue if they generate always certs with the fqdn in lowercase, although it's not what the GNU suite does...

Okay, I'll look for the Puppet code. When I first talked to Dominic about
the issue, he suggested that Puppet couldn't really be changed, which is
why I ended up changing pretty much everything else... but I didn't look at
Puppet myself, so I'll do that.

On Wed, Mar 26, 2014 at 7:55 PM, Daniel Lobato García <
notifications@github.com> wrote:

Good one, thanks for finding this bug!

Maybe I'm asking for too much but could you point to the code in Puppet
that generates the SSL certs, to make sure they always are created in
lowercase?

FQDNs are case insensitive from my 5 min investigation and fiddling with
/etc/hosts, so it's not an issue if they generate always certs with the
fqdn in lowercase, although it's not what the GNU suite does...

Reply to this email directly or view it on GitHubhttps://github.com//pull/1330#issuecomment-38755093
.

It's in a number of places, as the requirement for lower case files is persistent throughout the code.

https://github.com/puppetlabs/puppet/blob/master/lib/puppet/application/cert.rb#L215
https://github.com/puppetlabs/puppet/blob/master/lib/puppet/ssl/base.rb#L46-L47
https://github.com/puppetlabs/puppet/blob/master/lib/puppet/ssl/host.rb#L244-L245
https://github.com/puppetlabs/puppet/blob/master/lib/puppet/indirector/ssl_file.rb#L143

[test]

Merged as 0b2e3dc , thanks @anyaelise ! I'll make a note on the OPW application you've successfully submitted a contribution.

Excellent, thanks @elobato! And thanks as well @domcleal. It was a pleasure
working with you guys.

On Thu, Mar 27, 2014 at 6:54 AM, Daniel Lobato García <
notifications@github.com> wrote:

Closed #1330 #1330.

Reply to this email directly or view it on GitHubhttps://github.com//pull/1330
.